Skip to content

Enable fail-fast logit NaN detection in vLLM CI - #444

Open
tlrmchlsmth wants to merge 1 commit into
vllm-project:mainfrom
tlrmchlsmth:ci/raise-on-logit-nans
Open

Enable fail-fast logit NaN detection in vLLM CI#444
tlrmchlsmth wants to merge 1 commit into
vllm-project:mainfrom
tlrmchlsmth:ci/raise-on-logit-nans

Conversation

@tlrmchlsmth

Copy link
Copy Markdown
Member

Summary

  • set VLLM_RAISE_ON_LOGIT_NANS=1 on generated vLLM CI command steps
  • cover standard and AMD-specific generated jobs
  • preserve per-step overrides for tests that intentionally produce NaNs

This is the CI-side companion to vllm-project/vllm#50323. The vLLM flag implies NaN detection and raises when a nonzero count reaches output bookkeeping.

Duplicate check

Searched open ci-infra PRs for VLLM_RAISE_ON_LOGIT_NANS and NaN logits; no duplicate PR was found.

Tests

  • /Users/tms/code/vllm/.venv/bin/python -m pytest buildkite/tests -q - 62 passed
  • pre-commit run --files buildkite/pipeline_generator/buildkite_step.py buildkite/tests/pipeline_generator/test_step.py - passed

AI assistance was used for this change.

Set VLLM_RAISE_ON_LOGIT_NANS=1 on generated vLLM CI steps, including AMD jobs, while allowing explicit per-step overrides.

Co-authored-by: OpenAI Codex <codex@openai.com>
Signed-off-by: Tyler Michael Smith <tlrmchlsmth@gmail.com>
@NickLucche

Copy link
Copy Markdown
Member

/ci run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants